Skip to content

Issue 2809 fix #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 8 commits into from
Closed

Issue 2809 fix #36

wants to merge 8 commits into from

Conversation

PrakashDurlabhji
Copy link

@PrakashDurlabhji PrakashDurlabhji mentioned this pull request Jul 18, 2019
Closed
@PrakashDurlabhji
Copy link
Author

@veshu @codeMinter can anyone please review this?

@sexta13 sexta13 self-assigned this Jul 28, 2019
@sushilshinde
Copy link

@PrakashDurlabhji @sexta13 wrong target branch

@sushilshinde sushilshinde added the On Hold Blocked by another ticket / issue / etc. label Jul 29, 2019
@sushilshinde
Copy link

Use https://github.com/topcoder-platform/topcoder-react-utils/tree/qa-accessibility

@PrakashDurlabhji PrakashDurlabhji changed the base branch from develop to qa-accessibility July 29, 2019 11:36
@PrakashDurlabhji
Copy link
Author

@sushilshinde ready

@sexta13
Copy link
Collaborator

sexta13 commented Jul 29, 2019

@PrakashDurlabhji , your commit messes with a lot of things and none of them seem like the clear filter/save filter focus problem. Can you please review?

@PrakashDurlabhji
Copy link
Author

@sexta13 yes sure will raise new PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sexta13 sexta13 sexta13 left review comments

Assignees

@sexta13 sexta13

Labels
Accessibility On Hold Blocked by another ticket / issue / etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@PrakashDurlabhji @sushilshinde @sexta13 @crazyk07 @liuliquan @ThomasKranitsas @birdofpreyru